home *** CD-ROM | disk | FTP | other *** search
/ TeX 1995 July / TeX CD-ROM July 1995 (Disc 1)(Walnut Creek)(1995).ISO / tex-k / tex-k-archive.past / tex-k-archive.gz / tex-k-archive / 000162_simon@lia.di.epfl.ch_Mon Jan 17 12:39:03 1994.msg < prev    next >
Internet Message Format  |  1994-10-11  |  3KB
  1. Received: from liasun6.epfl.ch by cs.umb.edu with SMTP id AA22059
  2.   (5.65c/IDA-1.4.4 for <tex-k@cs.umb.edu>); Mon, 17 Jan 1994 12:39:03 -0500
  3. Received: by liasun6.epfl.ch (Smail3.1.28.1 #58)
  4.     id m0pLxum-0002P8C; Mon, 17 Jan 94 18:39 MET
  5. Message-Id: <m0pLxum-0002P8C@liasun6.epfl.ch>
  6. Date: Mon, 17 Jan 94 18:39 MET
  7. From: simon@lia.di.epfl.ch (Simon Leinen)
  8. To: tex-k@cs.umb.edu
  9. Subject: xdvik-1.4: SIGIO handling broken for POSIX systems
  10. In-Reply-To: <"swan.cl.cam.:121540:931022092439"@cl.cam.ac.uk>
  11. References: <"swan.cl.cam.:121540:931022092439"@cl.cam.ac.uk>
  12.  
  13. Hello,
  14.  
  15.   xdvik-1.4 includes a fix proposed by Martyn Johnson (see the
  16. referenced message to this list).  Unfortunately (1) the fix wasn't
  17. integrated correctly and (2) although the idea is good, the fix itself
  18. has a bug.  Martyn's fix was:
  19.  
  20. > Solution 2:
  21. > 
  22. >   Use "sigaction" rather than "signal", and set the SA_RESTART flag
  23. > explicitly.  I prefer this approach, because not all vendors may
  24. > follow Digital's approach of offering a whole family of alternative
  25. > compatibility modes.  A possible patch would be:
  26. > 
  27. > *** 1.1 1993/10/21 14:06:44
  28. > --- xdvi.c      1993/10/22 08:43:59
  29. > ***************
  30. > *** 2026,2032 ****
  31. > --- 2026,2036 ----
  32. >             if (dup2(socket, 0) == -1) perror(prog);
  33. >             socket = 0;
  34. >         }
  35. > + #ifdef SA_RESTART
  36. > +       (void) sigaction(SIGIO, (handle_intr, 0, SA_RESTART), NULL);
  37. > + #else
  38. >         (void) signal(SIGIO, handle_intr);
  39. > + #endif SA_RESTART
  40. >         (void) fcntl(socket, F_SETOWN, getpid());
  41. >         (void) fcntl(socket, F_SETFL, fcntl(socket, F_GETFL, 0) | FASYNC);
  42. >   }
  43.  
  44. but in xdvik/xdvi.c I see (lines 2029-2034):
  45.  
  46. #ifdef SA_RESTART
  47.         (void) sigaction(SIGIO, (handle_intr, 0, SA_RESTART), NULL);
  48. #else
  49.         (void) signal(SIGIO, handle_intr);
  50. #endif /* SA_RESTART */
  51.     (void) signal(SIGIO, handle_intr);
  52.  
  53. So there is one signal() call too much.  Probably this doesn't really
  54. matter.  The real problem is that the second parameter to sigaction()
  55. is wrong.  If I use this code under IRIX 5, I get spurious SIGIOs that
  56. terminate xdvi.  Here is a patch to xdvik-1.4 that fixes both bugs.
  57. -- 
  58. Simon.
  59.  
  60. *** xdvik/xdvi.c    1994/01/17 17:34:14    1.1
  61. --- xdvik/xdvi.c    1994/01/17 17:35:59    1.2
  62. ***************
  63. *** 2028,2036 ****
  64.       }
  65.   #ifdef SA_RESTART
  66. !         (void) sigaction(SIGIO, (handle_intr, 0, SA_RESTART), NULL);
  67.   #else
  68.           (void) signal(SIGIO, handle_intr);
  69.   #endif /* SA_RESTART */
  70. -     (void) signal(SIGIO, handle_intr);
  71.       (void) fcntl(socket, F_SETOWN, getpid());
  72.       (void) fcntl(socket, F_SETFL, fcntl(socket, F_GETFL, 0) | FASYNC);
  73. --- 2028,2041 ----
  74.       }
  75.   #ifdef SA_RESTART
  76. !     {
  77. !       struct sigaction sa;
  78. !       sa.sa_handler = handle_intr;
  79. !       sigemptyset (& sa.sa_mask);    
  80. !       sa.sa_flags = SA_RESTART;
  81. !       (void) sigaction(SIGIO, &sa, NULL);
  82. !     }
  83.   #else
  84.           (void) signal(SIGIO, handle_intr);
  85.   #endif /* SA_RESTART */
  86.       (void) fcntl(socket, F_SETOWN, getpid());
  87.       (void) fcntl(socket, F_SETFL, fcntl(socket, F_GETFL, 0) | FASYNC);